Skip to content

Xsup 67244/update py3 tools docker#44120

Merged
shmuel44 merged 8 commits into
masterfrom
XSUP-67244/update-py3-tools-docker
May 3, 2026
Merged

Xsup 67244/update py3 tools docker#44120
shmuel44 merged 8 commits into
masterfrom
XSUP-67244/update-py3-tools-docker

Conversation

@shmuel44
Copy link
Copy Markdown
Contributor

@shmuel44 shmuel44 commented Apr 30, 2026 •
edited
Loading

@shmuel44 shmuel44 requested a review from Ni-Knight as a code owner April 30, 2026 11:36
@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 30, 2026

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @marketplace-ai-reviewer start review - Initiate a full AI code review
  • @marketplace-ai-reviewer re-review - Incremental review for new commits

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CommonScripts/Scripts/DownloadAndArchivePythonLibrary
   DownloadAndArchivePythonLibrary.py37197%19
Packs/CommonScripts/Scripts/ExifRead
   ExifRead.py17194%35
Packs/CommonScripts/Scripts/ExtractDomainAndFQDNFromUrlAndEmail
   ExtractDomainAndFQDNFromUrlAndEmail.py70494%47, 50, 62, 142
Packs/CommonScripts/Scripts/ExtractDomainFromUrlFormat
   ExtractDomainFromUrlFormat.py641478%25–27, 61, 78–79, 81, 89–90, 92, 95–97, 102
Packs/CommonScripts/Scripts/ExtractFQDNFromUrlAndEmail
   ExtractFQDNFromUrlAndEmail.py721283%25–27, 49, 61, 75, 103–104, 107, 110–112
Packs/CommonScripts/Scripts/FetchIndicatorsFromFile
   FetchIndicatorsFromFile.py1383971%53, 71, 89, 92, 146, 203–204, 210–215, 220, 223–225, 229, 231–232, 244–245, 257, 259, 262–267, 269–275, 281–282
Packs/CommonScripts/Scripts/LanguageDetect
   LanguageDetect.py160100% 
Packs/CommonScripts/Scripts/ParseExcel
   ParseExcel.py31196%50
Packs/CommonScripts/Scripts/StixCreator
   StixCreator.py1604870%87–91, 93–98, 100–104, 193–196, 245, 273, 276–277, 279, 282–283, 286–288, 294, 296–298, 304–307, 309–310, 313–314, 316–317, 320–321, 333–334
Packs/CommonWidgets/Scripts/RSSWidget
   RSSWidget.py60886%36–39, 54, 57, 132–133
Packs/CommunityCommonScripts/Scripts/Json2HtmlTable
   Json2HtmlTable.py38781%35–36, 71, 86–89
Packs/FiltersAndTransformers/Scripts/Jmespath
   Jmespath.py21480%9–10, 21–22
TOTAL72413980% 

Tests Skipped Failures Errors Time
2 0 💤 0 ❌ 0 🔥 1.988s ⏱️

@shmuel44 shmuel44 requested review from BEAdi and removed request for Ni-Knight April 30, 2026 12:30
@shmuel44 shmuel44 added ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. labels Apr 30, 2026
@shmuel44 shmuel44 self-assigned this Apr 30, 2026
@marketplace-ai-reviewer marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Apr 30, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 30, 2026

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 30, 2026

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

marketplace-ai-reviewer
marketplace-ai-reviewer reviewed Apr 30, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@marketplace-ai-reviewer marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Thanks for your contribution to these packs. I reviewed the changes and noticed a few missing metadata fields that need to be updated across the packs. Specifically, please add "Cortex" to the empty keywords arrays, ensure the mandatory created key is present where missing, and provide a valid githubUser list for the community pack. Thanks for taking care of these updates!

Additionally, please address the following file-level notes:

  • Packs/CommonScripts/pack_metadata.json: The keywords list is empty. Since the vendor is Cortex XSOAR, please add "Cortex" to the keywords array.
  • Packs/CommonWidgets/pack_metadata.json: The keywords list is empty. Since the vendor is Cortex XSOAR, please add "Cortex" to the keywords array.
  • Packs/CommunityCommonScripts/pack_metadata.json: - The mandatory created key is missing from the pack metadata. Please add it.
  • The githubUser field is currently an empty list. Since this is a community-supported pack, please provide a valid list of GitHub usernames, or remove the field if it is not needed.
  • Packs/FiltersAndTransformers/pack_metadata.json: - The mandatory created key is missing from the pack metadata. Please add it.
  • The keywords list is empty. Since the vendor is Cortex XSOAR, please add "Cortex" to the keywords array.

@BEAdi, @shmuel44 please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@content-bot

This comment has been minimized.

Sign in to view
@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 30, 2026

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

@content-bot

This comment has been minimized.

Sign in to view
@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 30, 2026

This PR was automatically updated by a GitHub Action

  • CommonScripts pack version was bumped to 1.22.6.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 30, 2026

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: ad2b12f64ddfea25

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

@shmuel44 shmuel44 merged commit 71cb8b0 into master May 3, 2026
28 of 29 checks passed
@shmuel44 shmuel44 deleted the XSUP-67244/update-py3-tools-docker branch May 3, 2026 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marketplace-ai-reviewer marketplace-ai-reviewer marketplace-ai-reviewer left review comments

@BEAdi BEAdi BEAdi approved these changes

Assignees

@shmuel44 shmuel44

Labels

docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shmuel44 @content-bot @marketplace-ai-reviewer @BEAdi